Merge branch 'attribute-docs' into 'master'

entry: Document buildable support

See merge request GNOME/gtk!1643

entry: Document buildable support

entry: Add <attributes> support

Make entries support the same <attributes> syntax
as labels.

Closes: #1335

Move label attribute parser code

We want to reuse the parser for <attributes> in several
widgets, so move it to a shared place.

Merge branch 'cross-postinstall' into 'master'

meson: Don't execute post-install.py if cross compiling

See merge request GNOME/gtk!1636

Merge branch 'fix-event-match' into 'master'

shortcuttrigger: Don't trigger on key releases

See merge request GNOME/gtk!1642

Merge branch 'wip/chergert/fix-shortcut-param-order' into 'master'

widget: fix parameter ordering to match implementation

Closes #2586

See merge request GNOME/gtk!1641

shortcuttrigger: Don't trigger on key releases

This broke when the event type check in gdk_key_event_matches
was removed and replaced by a precondition that accepts both
key press and release events.

Add the check in gtk_keyval_trigger_trigger instead.

widget: fix parameter ordering to match implementation

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/2586

Replace most uses of <Primary> with <Control>

The only place where we are still using <Primary> is
in tests, to ensure we keep parsing it. Otherwise,
<Control> is now the preferred syntax.

Merge branch 'keymap-rework-2' into 'master'

Keymap rework 2

See merge request GNOME/gtk!1635

reftests: Comment out a test that fails only in ci

Fixing such only-in-ci failures is just too hard,
unfortunately.

Update migration guide

Rewrite the sections about keymaps and menus, and add
hints about modifiers.

accelgroup: Small documentation updates

Mention GtkShortcutTrigger, and remove references
to <Release>.

Reshuffle keymap docs

Since GdkKeymap api is on longer public, move relevant
documentation to the long description.

Drop gtk_accelerator_set_default_mod_mask

Our new approach to modifiers works with a fixed set,
there is really no need to customize the modifier
masks if the backends are all supposed to deliver
the same modifiers.

gdk: Remove gdk_keymap_get_modifier_mask

This function is no longer public and no longer used.

Remove uses of modifier intents in gdkevents.c

This removes the use of the context menu and shift group
intents in gdkevents.c. If it turns out to be important,
we need to introduce vfuncs for gdk_event_triggers_context_menu
and gdk_event_matches.

display: Remove gdk_display_get_modifier_mask

This is not used anymore.

gtk: Drop gtk_widget_get_modifier_mask

This is not used anymore.

gtk: Stop using modifier intents

Reviewing the existing settings, the only backend with
some differences in the modifier intent settings is OS X,
and we would rather have that implemented by interpreting
the existing modifiers in the appropriate way.

                X11      Wayland  Win32    OS X

primary         ctrl     ctrl     ctrl     mod2
mnemonic        alt      alt      alt      alt
context menu    -        -        -        ctrl
extend sel      shift    shift    shift    shift
modify sel      ctrl     ctrl     ctrl     mod2
no text         alt|ctrl alt|ctrl alt|ctrl mod2|ctrl
shift group     varies   -        -        alt

GTK now uses the following modifiers:

primary         ctrl
mnemonic        alt
extend sel      shift
modify sel      ctrl
no text         alt|ctrl

The context menu and shift group intents were not used
in GTK at all.

Update tests to no longer expect <Primary> to roundtrip
through the accelerator parsing and formatting code.

keymap: Remove virtual modifier mapping functions

These are not used anymore.

Drop the vfuncs and their implementations in the
broadway, x11, wayland and win32 backends as well.

gdk: Make GdkKeymap a private api

We have replacement apis in GdkDevice and GdkDisplay.

Merge branch 'misc-meson-fixes' into 'master'

Reduce useless relinking on configure and fix check for buildtype arguments

See merge request GNOME/gtk!1614

Update Dutch translation

imcontextsimple: Disable some win32-only code

This code needs to be redone differently, since keymaps are no
longer going to be exposed. There should really not be this much
ifdef-ed backend-specific code here anyway. Or any, really.

gdk: Clean up GdkModifierType

Remove MOD2..MOD5. Backends are expected to just set
the named modifiers.

gdk: Redo key events

Add all of the keyboard translation results in the key event,
so we can translate the keyboard state at the time the event
is created, and avoid doing state translation at match time.

We actually need to carry two sets of translation results,
since we ignore CapsLock when matching accelerators, in
gdk_event_matches().

At the same time, drop the scancode field - it is only ever
set on win32, and is basically unused in GTK.

Update all callers.

gtk: Remove handling of virtual modifiers

These are going away. We expect to just have
named modifiers in modifier masks now, so we
longer juggle 'real' and 'virtual' modifiers.

events: Stop doing elaborate virtual modifier handling

These are going away in the GdkModifierType cleanup.
Just compare the modifiers we got.

broadway: Stop using MOD2..MOD5

These are going away in the GdkModifierType cleanup.

win32: Stop using MODx modifiers

The win32 backend is using GDK_MOD2_MASK for AltGr,
so define GDK_MOD2_MASK locally to keep this working,
but remove any mention of GDK_MOD3_MASK,...,GDK_MOD5_MASK.

Update Ukrainian translation

Update Ukrainian translation

Merge branch 'fno-common' into 'master'

Add a missing extern, to fix building with -fno-common

See merge request GNOME/gtk!1631

wayland: Stop setting MOD2..MOD5

These are going away in the GdkModifierType cleanup.

gdk: Remove reserved bit from GdkModifierType

This really has no purpose and obscures the header.

gtk: Stop using GDK_MODx_MASK

These are never used in practice, and we never want to
see them in the UI, so stop supporting them. This is
in preparation for cleaning up GdkModifierType.

Rename GDK_MOD1_MASK to GDK_ALT_MASK

We've hardcoded Mod1 = Alt for a long time, there is
no need to keep the confusing naming around anymore.

tests: Stop using gtk_accelerator_set_default_mod_mask

This is not doing anything useful here.

testsuite: Stop using keymap apis

gtk: Use display apis for keymap mapping

This removes the last uses of GdkKeymap from GTK.

gtk: Stop using keymap apis

Stop using most keymap apis in GTK. Only a few calls
related to virtual modifiers are left.

x11: Notify new device properties

wayland: Notify new device properties

Move keymap mapping functions to GdkDisplay

We want to stop exposing GdkKeymap, so this
functionality needs a new home.

Move GdkKeymapKey to gdktypes.h

We are going to use this type in a different header.

gdk: Move modifier mask api to GdkDisplay

GdkKeymap is on the way out.

gdk: Add keymap properties to GdkDevice

GdkKeymap is on the way out.

passwordentry: Show caps lock warning more

Show the caps lock warning regardless of whether we
have the peek icon or not.

Drop some no longer used code

cell renderer accel: Use gdk_event_get_match

Add gdk_event_get_match

This is a counterpart to gdk_event_matches() that can
be used to obtain a shortcut matching an event.

Add gdk_event_matches

Move the elaborate key event matching code from
GtkShortcutTrigger to GdkEvent, which greatly reduces
the amount of keymap api use outside of GDK.

meson: Don't execute post-install.py if cross compiling

Add a missing extern, to fix building with -fno-common

With C compilers defaulting to -fcommon, this isn't an issue, but
upcoming compilers (GCC 10 and Clang 11) will default to -fno-common,
ending up with duplicate definitions of these variables.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1630

gtk-demo: Fix a crash

Using the assistant demo repeatedly was crashing.

Merge branch 'matthiasc/for-master' into 'master'

shortcuts: documentation fixes

See merge request GNOME/gtk!1628

Merge branch 'matthiasc/for-master' into 'master'

shortcuts: documentation fixes

See merge request GNOME/gtk!1628

shortcuts: documentation fixes

Update symbol lists, and add examples for parsing triggers
and actions.

Update POTFILES.skip

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1626

colorbutton: Don't add a customize menu

The color swatch in the button does not need a
context menu. Turn it off.

colorbutton: Fix focus

The colorbutton contains a button which contains a colorswatch.
We want the focus to go straight to the button, nowhere else,
so mark the swatch as !can-focus.

Adapt tests to match.

demos: Remove can-focus from ui files

Merge branch 'scrollbar-a11y' into 'master'

Add GtkScrollbarAccessible

See merge request GNOME/gtk!1623

scrollbar: Be more defensive

Somehow the scrollbar accessible manages to call
gtk_scrollbar_get_adjustment when priv->range is NULL.
Survive taht.

Merge branch 'shortcut-action-parse' into 'master'

Shortcut action parse

See merge request GNOME/gtk!1625

testsuite: Add tests for parsing shortcut actions

Add an api to parse shortcut actions

This is so we can test this code. While doing it
I also fixed the missing else that was caused it
not to work.

Add GtkScrollbarAccessible

When GtkScrollbar stopped being a GtkRange,
GtkRangeAccessible stopped working for scrollbars.

Update a11y tests to match.

Merge branch 'wip/carlosg/fix-surrounding-text-master' into 'master'

imwayland: Clamp the surrounding string end correctly

Closes #2565

See merge request GNOME/gtk!1621

Merge branch 'focus-test' into 'master'

Focus test

See merge request GNOME/gtk!1622

imwayland: Clamp the surrounding string end correctly

In the paths where len > MAX_LEN and cursor/anchor are separated by
at least MAX_LEN from text edges, we were clamping the right end of
the surrounding string at MAX_LEN. Oops.

This end anchor may go as far as the string length, although just
up to len - MAX_LEN in real terms (due to the condition above that
caches cursor/anchor positions being near enough the text end).

Closes: https://gitlab.gnome.org/GNOME/gtk/-/issues/2565
(cherry-picked from commit d7fb15c822eabe22ffda19892153d69b0f4fc2f3)

testsuite: Add a big focus chain testcase

testsuite: Try harder to record a full focus chain

It is hard to avoid widgets with the same name in a
large ui file - try harder to record a full focus chain
before decide that we've wrapped, by including the widget
address in the comparison. Note that we don't include
the addresses in the generated output, since that would
make expected output vary from run to run.

Merge branch 'gir-fixes' into 'master'

Annotation fixes

See merge request GNOME/gtk!1616

Merge branch 'move-gdkgrab-decl' into 'master'

gdk: Move GdkGrabStatus and GdkGrabOwnership decl.

See merge request GNOME/gtk!1615

Annotation fixes

Match the arguments name in the declaration and the definition, and make
sure we have the proper transfer annotations.

gdk: Move GdkGrabStatus and GdkGrabOwnership decl.

With the removal of `gdk_seat_grab` we do not need to keep the
definitions of `GdkGrabStatus` and `GdkGrabOwnership` public.

Move those definitions to become internal only.

meson: Fix check for builtype arguments

`get_option('buildtype')` will return `'custom'` for most combinations
of `-Doptimization` and `-Ddebug`, but those two will always be set
correctly if only `-Dbuildtype` is set. So we should look at those
options directly.

For the two-way mapping between `buildtype` and `optimization`
+ `debug`, see this table:
https://mesonbuild.com/Builtin-options.html#build-type-options

meson: Reduce useless relinking on reconfigure

When we reconfigure, `configure_file()` is called again, and
`*.gresource.xml` files are regenerated, which causes many (all?)
binaries to be relinked. Now we only write those out if the contents
actually changed (or if the output didn't already exist).

This is exactly what Meson already does with `configure_file()` when
`command:` is not used.

While we're at it, also do the same for `gen-c-array.py` and
`gentypefuncs.py` for completeness. Now even if the input to those
changes, re-building of those custom targets may not result in
relinking if the outputted C files have the same contents.

shortcutcontroller: fix transfer full annotation

window: Don't leak toplevel layout

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1611

wayland: Add a comment about keyboard state handling

Clarify a point that took me a few hours to fully track
down, so lets preserve what I found for the next poor
sould coming this way.

a11y tests: Update for focus state changes

a11y: Set focus-related states properly

Entries and menubuttons are no longer focusable themselves,
they have focusable children. Since we don't have accessible
objects for those, transfer the focus-related state (focusable
and focused) from the children to the main accessible object.

window: tie focus to surface state changes

We don't get a focus-out on the event controller, when
the surface is losing keyboard focus, since we are not
moving our focus to some other widget, so we are never
unsetting the mnemonics-visible property. Do that in
response to surface state changes instead.

Merge branch 'cherry-picked-wayland-keyboard-fix' into 'master'

gdk/wayland: Clear modifiers when we lose keyboard focus

Closes #2112

See merge request GNOME/gtk!1610

gdk/wayland: Clear modifiers when we lose keyboard focus

When we `Alt+Tab` away from a GTK application, it loses keyboard focus.
If we don't clear the modifiers, events from other devices that we
receive while unfocused will assume `Alt` is still pressed. This results
in e.g. Firefox navigating through the history instead of scrolling the
page when using the mouse wheel on it.

We don't get any information about modifiers while we are missing
keyboard focus, so assuming no modifiers are active is the best we can
do.

The shell sends us a modifier update immediately before we regain
keyboard focus, so the state shouldn't get out of sync.

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/2112

Merge branch 'wip/smcv/libdir-independent-valgrind-suppressions' into 'master'

supp: Use a single suppressions file for lib, lib64 and multiarch

See merge request GNOME/gtk!1603

Merge branch 'im-context-wayland-fixes' into 'master'

Im context wayland fixes

See merge request GNOME/gtk!1605

imcontexsimple: Move Compose file loading to class_init

This ensures that it is done only once.

imcontextsimple: Simpify Compose file loading

We are not loading the Compose file for individual contexts,
it just gets added to a global list. So don't pass an im context
along. This will let us move the loading out of the initialization
of individual contexts, and only do it once.

imcontextsimple: Load ~/.Compose unconditionally

There is nothing display-specific in the ~/.Compose file,
so we can just try to load it without looking at the
display we are on.

imcontextsimple: Add a profiler mark for IO

We are loading a file here, thankfully in a thread, but
we do it every time an entry is created. Add a profiler
mark, to make this visible.

supp: Use a single suppressions file for lib, lib64 and multiarch

In addition to the traditional library directory lib and the 64-bit
multilib directory lib64, this will cover Debian-style multiarch
(lib/x86_64-linux-gnu etc.), Arch Linux 32-bit (lib32), x32 and
various others.

Signed-off-by: Simon McVittie <smcv@debian.org>

Merge branch 'fix-menu-flicker' into 'master'

Fix crossing event generation

Closes #2536

See merge request GNOME/gtk!1601